Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom and main class can be set at the same time #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Custom and main class can be set at the same time #13

wants to merge 1 commit into from

Conversation

BigOHenry
Copy link

Custom class (set with ->setClass();) can be set at the same time with main classes (selectize form-control text). They are important to working.

Custom class (set with ->setClass();) can be set at the same time with main classes (selectize form-control text). They are important to working.
@BigOHenry
Copy link
Author

Hello, any motivation to do something with this? :-) At least answer :-)

@Olicek
Copy link
Owner

Olicek commented Apr 5, 2018

Ahoj, promiň, nějak jsem to přehlídl. Co je přesně cílem? Aby ti tam vždy (i když zadáš $this->options['class']) zůstala ta třída selectize? To ale pravděpodobně nechceš ve chvíli, kdy používáš víc selectize inputů na stránku nebo je potřebuješ kvůli něčemu jinému rozlišit jinou třídou. Viz.: https://github.com/Olicek/form-selectize/blob/master/docs/en/javascript.md.

Díky za dovysvětlení 😄

@BigOHenry
Copy link
Author

Hele asi máš pravdu :) Teď když na to koukám s odstupem času, tak se mi to jeví celkem jako blbost. Jak říkáš, pak by nebyla možnost nastavit třídu jakou chci pro selectize :-)

Můžu mít jeden dotaz trochu z jiného soudku? Mám problém s tím, že mi u full mode selectize nefunguje validace, respektive setRequired. Pole nezobrazí hlášku když není vyplněné. Pro režim select to funguje v pohodě. Nemáš nějaký typ jak to vyřešit? Díky moc :-)

@BigOHenry
Copy link
Author

BigOHenry commented Apr 6, 2018

Tak už vím proč jsem to tenkrát řešil. V tomto stavu totiž nefunguje full mode .

V tomto režimu se ignoruje setování class přes ->setAttribute() a musí být nastavena přes selectize settings. Což mi přijde dost nešťastné...

Proto jsem dělal tuto úpravu. Takhle jsou pak 2 možné varianty jak se k tomu musí přistupovat:

  1. mode select - musí se nastavit class přes setAttribute - selectize form-control
  2. full mode - musí se nastavit class - selectize form-control text - je v tomto stavu nefunkční (setAttribute se ignoruje - nebo později přepíše), musí se nastavit class přes selectize settings

Další věcí, která mi na tom vadí je ta, že pokud chci nechat selectize třídu tu default a zároveň chci nastavit nějakou další, tak tu třídu selectize form-control nebo selectize form-control text musím nastavit stejně, jinak to nebude funkční a tím pádem na to musím pořád myslet.

Když už, tak bych tu třídu na určení selectize dal definovat jiným parametrem aby i po nastavení dalších tříd bylo funkční.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants